Fit chat approvals and the Agent inbox to a phone-width screen by Birfy · Pull Request #481 · ThinkFlowLab/FlyBot

Birfy · 2026-10-20T03:01:06Z

Fixes #362

Step 1 only, per the Owner decision (2026-10-20): a phone-width layout for the chat, the held-reply Approve / Reject buttons and the Agent inbox. No PWA, push or native app.

What. One new style block, core/src/client/mobile.js (MOBILE_CSS, all under @media (max-width:640px), installed last in the shell group), plus one phone rule in the social hold card:

Held-reply and floating approval cards: Reject / Approve share the card width as 40px buttons, Approve filled with the accent; the held card uses the full message width. Social's hold card gets 40px Approve / Reject too.
The floating approval stack and the chat pointers span the screen beside DSH's 56px icon rail (DSH already collapses the sidebar below 1024px), so the top + and the account stay reachable.
Inbox cards put their buttons on a row under the text (36px) and wrap title/reason to two lines instead of truncating to "Run npm i…". The inbox dialog already went full-screen at this width.

Why. At 400px the inbox card's buttons squeezed the title to a few characters, the held-reply buttons were ~24px browser defaults, and the pointer covered the top +.

Test. core/test/mobile-layout.test.js (rules sit in one media query, button sizes, rail clearance, inbox wrap, install order and bundle) and a social test for the hold-card rule. ARCH_DOCS_BASE=origin/main TZ=UTC npm run verify passes (core 1062, social 292, tree 80, timeline 80, scripts 88, interop 37, live-e2e 16; 0 failures, 0 skipped). Checked by eye on a local mock dsh web (npm harness 0.2.0-rc.2, mock model) in headless Chrome at 400px and 360px: chat, inbox, held card and floating approvals. Wider screens are unchanged (every rule sits in the max-width query).

Risk tier: T1 (estimate; the controller sets the label)

…FlowLab#362) Step 1 of ThinkFlowLab#362 (owner decision 2026-10-20): below 640px the held-reply and floating approval buttons are 40px touch targets (Approve filled), the floating approval stack and chat pointers sit beside the 56px icon rail, and inbox cards wrap their buttons under the text. Wider screens are unchanged. No PWA, push or native app.

Birfy · 2026-10-20T03:03:25Z

Self-review: PASS
Tip SHA: 774ba97
Ran:

ARCH_DOCS_BASE=origin/main TZ=UTC npm run verify: pass (core lint + arch-docs OK, plugin lint, build-client --check, ds-bot-patches check; core 1062, social 292, tree 80, timeline 80, scripts 88, interop 37, live-e2e 16; 0 fail, 0 skipped)
node scripts/ds-bot-patches.mjs generate then check: ok; node .github/scripts/changelog.mjs check: ok
CI at tip: tests-ok, test, plugins (social/tree/timeline), package all green; risk tier (report only) T2
Local mock dsh web (npm harness DSH 0.2.0-rc.2, mock model) in headless Chrome at 400px and 360px: chat, Agent inbox card, held-reply card and floating approval stack checked by eye before and after
Design rules re-read against the diff: approval stays on explicit UI buttons only (no new path), English UI, one top + menu kept reachable, scope limited to step 1 (no PWA/push/native), wider screens unchanged (all rules in one max-width query)
Not run:
Real phone browsers (iOS Safari / Android Chrome); a live friend's held reply through Bot Social (held card drawn from fixture markup on the bench)
The user's Mac instances and the production Bot Social server (not touched)

# Conflicts: # patches/ds-bot/0008-client-bundle.patch

Birfy · 2026-10-20T03:18:28Z

Self-review: PASS
Tip SHA: 44fe62c
Ran:

Merged origin/main (674bcd7, with Search Composio's app list and show each picked app's sign-in status #479 and Add a Dockerfile, compose file and guide to run FlyBot in a container #482) into the branch (merge commit, no rebase, no force-push). Only conflict: generated patches/ds-bot/0008-client-bundle.patch; resolved per AGENTS.md conflict recipe (took main's patches/ds-bot, npm run build in core, ds-bot-patches.mjs generate then check: ok). Both behaviors kept: Search Composio's app list and show each picked app's sign-in status #479 Composio app search and Add a Dockerfile, compose file and guide to run FlyBot in a container #482 Docker files intact, MOBILE_CSS still installed and in client.js.
ARCH_DOCS_BASE=origin/main TZ=UTC npm run verify on Node 24.21.0: pass (arch-docs OK; core 1069, social 292, tree 80, timeline 80, scripts 93, interop 37, live-e2e 16; 0 fail, 0 skipped)
CI at tip: tests-ok, test, plugins (social/tree/timeline), package all green; risk tier (report only) T2
Branch diff vs main (excluding generated files) unchanged from the previous review: 8 files, +112; design rules re-checked (approvals only via explicit buttons, English UI, top + reachable, step 1 scope only)
Earlier, at 774ba97: headless Chrome on a local mock dsh web at 400px and 360px (chat, inbox card, held-reply card, floating approvals)
Not run:
Screenshots not repeated after the merge (the merge touched no Mobile: PWA first, then native — approvals and reminders on the phone #362 code)
Real phone browsers (iOS Safari / Android Chrome); a live friend's held reply through Bot Social
The user's Mac instances and the production Bot Social server (not touched)

# Conflicts: # patches/ds-bot/0008-client-bundle.patch # plugins/social/test/social.test.js

Birfy · 2026-10-20T04:16:23Z

Self-review: PASS
Tip SHA: 22b6d8d
Ran:

Merged origin/main (a7adfe0, with Label the model each turn ran on in the Timeline and each Bot row #480, Count another home's Bot answer in face rooms; a friend Bot's @all wakes none of mine #487 and Face rooms: one room lead picked by the room creator, cross-home roster with roles #488) into the branch (merge commit on top of 44fe62c, no rebase, no force-push). Conflicts: generated patches/ds-bot/0008-client-bundle.patch, resolved per AGENTS.md conflict recipe (took main's patches/ds-bot, npm run build in core, ds-bot-patches.mjs generate then check: ok); plugins/social/test/social.test.js, kept both sides' tests (Mobile: PWA first, then native — approvals and reminders on the phone #362 phone-width hold card test and the Face rooms: selectable room lead (总负责人) picked by the room creator, cross-home roster with roles, one plan per request #476 room lead tests).
ARCH_DOCS_BASE=origin/main TZ=UTC npm run verify on Node 24.21.0: pass (arch-docs OK, plugin lint, build-client --check, ds-bot-patches check; core 1082, social 297, tree 80, timeline 83, scripts 93, interop 37, live-e2e 16; 0 fail, 0 skipped)
node .github/scripts/changelog.mjs check: ok (changelog.d/362-mobile-layout.added.md kept; CHANGELOG.md and ds-bot/ untouched)
CI at tip: tests-ok, test, plugins (social/tree/timeline), package all green; risk tier (report only) T2
Branch diff vs main (excluding generated files) unchanged from the previous review: 8 files, +112; design rules re-checked (approvals only via explicit buttons, English UI, top + reachable, step 1 scope only)
Earlier, at 774ba97: headless Chrome on a local mock dsh web at 400px and 360px (chat, inbox card, held-reply card, floating approvals)
Not run:
Screenshots not repeated after the merge (the merge touched no Mobile: PWA first, then native — approvals and reminders on the phone #362 code)
Real phone browsers (iOS Safari / Android Chrome); a live friend's held reply through Bot Social
The user's Mac instances and the production Bot Social server (not touched)

…ThinkFlowLab#481/ThinkFlowLab#501/ThinkFlowLab#505) Keep ThinkFlowLab#499 composer-label / data-placeholder docs with ThinkFlowLab#481 phone hold-card targets. Regenerated client bundle patch after merge.

…inkFlowLab#481/ThinkFlowLab#501/ThinkFlowLab#505) Keep ThinkFlowLab#495 reply_only People-with-access docs from main with ThinkFlowLab#496 already-have copy and Remove confirm.

…owLab#481/ThinkFlowLab#501/ThinkFlowLab#505) Regenerated client bundle patch after merge; auto-merged social sources keep ThinkFlowLab#498 friend Bot name formatter with ThinkFlowLab#501/ThinkFlowLab#505.

…ThinkFlowLab#481/ThinkFlowLab#501/ThinkFlowLab#505) Regenerated client bundle patch after merge.

…wLab#481/ThinkFlowLab#501/ThinkFlowLab#505) Regenerated ds-bot patches (0005/0008) after merge.

ThinkFlowLab#481/ThinkFlowLab#501/ThinkFlowLab#505) Regenerated ds-bot patches after merge.

…nkFlowLab#481/ThinkFlowLab#501/ThinkFlowLab#505) Regenerated ds-bot patches after merge.

Birfy added agent-pr Pull request opened by an agent enhancement New feature or request area:ui Web/desktop UI and copy labels Oct 20, 2026

Birfy self-assigned this Oct 20, 2026

Birfy mentioned this pull request Oct 20, 2026

Mobile: PWA first, then native — approvals and reminders on the phone #362

Closed

Birfy added the awaiting-approval PR is green and waits for the maintainer's approval (T2/T3) or a review PASS at its tip label Oct 20, 2026

Birfy added this pull request to the merge queue Oct 20, 2026

github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Oct 20, 2026

Merge remote-tracking branch 'origin/main' into feat/362-mobile-layout

44fe62c

# Conflicts: # patches/ds-bot/0008-client-bundle.patch

Birfy added this pull request to the merge queue Oct 20, 2026

github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Oct 20, 2026

Merge remote-tracking branch 'origin/main' into feat/362-mobile-layout

22b6d8d

# Conflicts: # patches/ds-bot/0008-client-bundle.patch # plugins/social/test/social.test.js

github-actions Bot added the risk:T2 Risk tier T2 (host/plugin code or an unnamed path), set by the controller from the diff label Oct 20, 2026

Birfy added this pull request to the merge queue Oct 20, 2026

Merged via the queue into ThinkFlowLab:main with commit 2c2cabc Oct 20, 2026
8 checks passed

Birfy deleted the feat/362-mobile-layout branch October 20, 2026 06:47

Birfy mentioned this pull request Oct 20, 2026

Polish invites and friend chats: friend Bot placeholder, one invite control, Add friend auto-close, Sharing hint #504

Merged

Birfy mentioned this pull request Oct 20, 2026

Social: neutral 'You already have …' on a re-accepted invite; clearer Remove confirm #503

Merged

Birfy mentioned this pull request Oct 20, 2026

Name a friend's Bot one way and approval cards by the receiving person #502

Merged

Birfy added a commit to Birfy/FlyBot that referenced this pull request Oct 20, 2026

Merge origin/main into feat/473-schedule-last-run-retry (resolve after …

0c2f52a

…ThinkFlowLab#481/ThinkFlowLab#501/ThinkFlowLab#505) Regenerated client bundle patch after merge.

Birfy mentioned this pull request Oct 20, 2026

Show each scheduled task's last run, and Retry a failed one #494

Merged

Birfy added a commit to Birfy/FlyBot that referenced this pull request Oct 20, 2026

Merge origin/main into feat/355-email-to-self (resolve after ThinkFlo…

8eff14c

…wLab#481/ThinkFlowLab#501/ThinkFlowLab#505) Regenerated ds-bot patches (0005/0008) after merge.

Birfy mentioned this pull request Oct 20, 2026

Email yourself when your Bots finish, fail or wait for approval #493

Closed

Birfy added a commit to Birfy/FlyBot that referenced this pull request Oct 20, 2026

Merge origin/main into feat/469-waiting-reminder-toggles (resolve after

79e6a40

ThinkFlowLab#481/ThinkFlowLab#501/ThinkFlowLab#505) Regenerated ds-bot patches after merge.

Birfy mentioned this pull request Oct 20, 2026

Remind about approvals waiting 15 min, switch each notification kind, one notification per chat #492

Merged

Birfy added a commit to Birfy/FlyBot that referenced this pull request Oct 20, 2026

Merge origin/main into feat/471-inbox-failed-turns (resolve after Thi…

2a737a1

…nkFlowLab#481/ThinkFlowLab#501/ThinkFlowLab#505) Regenerated ds-bot patches after merge.

This was referenced Oct 20, 2026

Agent inbox: show failed turns and a latest-activity line on each card #491

Merged

Group details panel: approval card buttons overflow off the left edge, 拒绝 unreachable at 1440 px #593

Closed

Birfy removed awaiting-approval PR is green and waits for the maintainer's approval (T2/T3) or a review PASS at its tip agent-pr Pull request opened by an agent labels Oct 21, 2026

Navigation Menu

Repository navigation

Fit chat approvals and the Agent inbox to a phone-width screen - #481

Fit chat approvals and the Agent inbox to a phone-width screen#481
Birfy merged 3 commits into
ThinkFlowLab:mainfrom
Birfy:feat/362-mobile-layout

Birfy commented Oct 20, 2026

Uh oh!

Birfy commented Oct 20, 2026

Uh oh!

Birfy commented Oct 20, 2026

Uh oh!

Birfy commented Oct 20, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

Birfy commented Oct 20, 2026

Uh oh!

Birfy commented Oct 20, 2026

Uh oh!

Birfy commented Oct 20, 2026

Uh oh!

Birfy commented Oct 20, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant